Skip to content

test(integration): cover remote & host build, CSS bundling and shared deps#406

Merged
gioboa merged 2 commits intomodule-federation:mainfrom
joshuaellis:poc/integration-tests
Feb 18, 2026
Merged

test(integration): cover remote & host build, CSS bundling and shared deps#406
gioboa merged 2 commits intomodule-federation:mainfrom
joshuaellis:poc/integration-tests

Conversation

@joshuaellis
Copy link
Contributor

@joshuaellis joshuaellis commented Feb 13, 2026

Description

Introduces an integration test suite that builds real Vite fixtures through the plugin and asserts on the Rollup output artifacts (chunks, assets, manifests, HTML) covering the four critical build scenarios: basic remote builds, host builds, CSS asset tracking in manifests, and shared dependency handling

Suite What it verifies
build.test.ts Remote entry generation, federation runtime init, exposed module output, manifest generation
host-build.test.ts Remote imports transformed to loadRemote(), hostInitInjectLocation modes (html vs entry), initializeSharing() presence
css-manifest.test.ts CSS/JS assets tracked per-expose in manifest, bundleAllCSS option
shared-deps.test.ts loadShare() rewriting, singleton/requiredVersion config in remoteEntry, import: false errors, shared deps in manifest

Adds a CI workflow that runs the integration tests against a Vite version matrix (5, 6, and 7) – integration-test.yml

@joshuaellis joshuaellis force-pushed the poc/integration-tests branch 3 times, most recently from 67b4318 to 2e03ed9 Compare February 13, 2026 13:55
Copy link
Collaborator

@gioboa gioboa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Validate compatibility across different Vite versions is something really valuable, happy to merge it. Thanks

@joshuaellis joshuaellis force-pushed the poc/integration-tests branch from 5721297 to 83e37a9 Compare February 13, 2026 16:35
@joshuaellis joshuaellis changed the title test(integration): add basic remote build tests as poc test(integration): cover remote & host build, CSS bundling and shared deps Feb 16, 2026
@joshuaellis joshuaellis force-pushed the poc/integration-tests branch from 252d0ab to cb3a7f9 Compare February 16, 2026 15:16
@joshuaellis joshuaellis marked this pull request as ready for review February 16, 2026 15:17
@joshuaellis joshuaellis requested a review from gioboa February 16, 2026 15:18
@joshuaellis
Copy link
Contributor Author

I've updated the description to explain what more this adds as a PR, I imagine we'll add more tests when looking at vite@7.1.7+

@joshuaellis
Copy link
Contributor Author

I'd quite like to get this PR landed @gioboa so i can write an integration test to cover this issue #274, I've got an upstream PR made to help alleviate the issue as well here – module-federation/core#4454

with that in mind, do you have any questions or feedback on it's approach?

Copy link
Collaborator

@gioboa gioboa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

with that in mind, do you have any questions or feedback on it's approach?

it looks awesome to me 👏
thanks @joshuaellis

@gioboa gioboa merged commit 9db8810 into module-federation:main Feb 18, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants